Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

download_sysext: check for download failure #24

Merged
merged 2 commits into from
Nov 16, 2023

Conversation

dongsupark
Copy link
Member

Define DownloadFailed status, and check for errors of HTTP client.
Check for HTTP status codes for immediate failures in download_and_hash, for now only 403 Forbidden and 404 Not found.

Fixes #14.

Define DownloadFailed status, and check for errors of HTTP client.
@dongsupark dongsupark force-pushed the dongsu/status-download-failed branch 2 times, most recently from dfe6c92 to 27b6fc9 Compare October 31, 2023 15:58
src/download.rs Outdated Show resolved Hide resolved
src/bin/download_sysext.rs Show resolved Hide resolved
@dongsupark dongsupark force-pushed the dongsu/status-download-failed branch from 27b6fc9 to c3fc2b3 Compare November 7, 2023 17:01
@dongsupark dongsupark marked this pull request as ready for review November 7, 2023 17:01
src/download.rs Outdated Show resolved Hide resolved
src/download.rs Outdated Show resolved Hide resolved
Check for HTTP status codes for immediate failures in download_and_hash,
for now only 403 Forbidden and 404 Not found.
@dongsupark dongsupark force-pushed the dongsu/status-download-failed branch from c3fc2b3 to d608308 Compare November 14, 2023 11:48
@dongsupark dongsupark merged commit f122a63 into trunk Nov 16, 2023
1 check passed
@dongsupark dongsupark deleted the dongsu/status-download-failed branch November 16, 2023 11:32
dongsupark added a commit to flatcar/scripts that referenced this pull request Nov 16, 2023
Update ue-rs to 0.1.0-r3, 2023-11-16.
Pulls in flatcar/ue-rs#23 and
flatcar/ue-rs#24.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PackageStatus should have a DownloadFailed state
3 participants